Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docker-orchagent]: Add vrfmgrd to supervisorctl #2055

Merged
merged 2 commits into from
Sep 20, 2018

Conversation

marian-pritsak
Copy link
Collaborator

Signed-off-by: Marian Pritsak marianp@mellanox.com

Signed-off-by: Marian Pritsak <marianp@mellanox.com>
@marian-pritsak
Copy link
Collaborator Author

Depends on sonic-net/sonic-swss#621

@lguohan
Copy link
Collaborator

lguohan commented Sep 18, 2018

also need this one in vs docker.

@lguohan
Copy link
Collaborator

lguohan commented Sep 18, 2018

Signed-off-by: Marian Pritsak <marianp@mellanox.com>
@lguohan
Copy link
Collaborator

lguohan commented Sep 19, 2018

can you include in this PR?

sonic-net/sonic-swss#621

@lguohan lguohan merged commit 8a5e6ac into sonic-net:master Sep 20, 2018
judyjoseph added a commit that referenced this pull request Mar 20, 2022
9968d60 (HEAD -> 202111, origin/202111) [sonic-package-manager] do not mod_config for whole config db when setting init_cfg (#2055)
4b3d53f [generate_dump] exclude mft and mlx folders from /etc (#2072)
51d92ae Validation check correction while adding a member to PortChannel (#2078)
6a43306 [techsupport] Added a lock to avoid running techsupport in parallel (#2065)
44cfdd9 Try get port operational speed from STATE DB (#2030)
45ea623 Fix sonic-installer failure due to missing import
liorghub added a commit to liorghub/sonic-buildimage that referenced this pull request Mar 23, 2022
Update sonic-utilities submodule to include below commits:
510a9b9 Add date to show version (sonic-net#2086)
61b1396 [sfpshow][recycle_port] sfpshow script needs to skip recycle ports (sonic-net#2109)
4681697 Support command `show ip fib` and `show ipv6 fib` (sonic-net#2100)
2a982a1 Update `config mirror_session` CLI to support heximal gre type value (sonic-net#2095)
b25f1e1 [generic-config-updater] Add caclrule validator (sonic-net#2103)
968900c [sonic-package-manager] do not mod_config for whole config db when setting init_cfg (sonic-net#2055)
bf55ceb [nvgre] Added auto-generated CLI plugins and test for NVGRE Tunnel feature (sonic-net#1915)
liat-grozovik pushed a commit that referenced this pull request Mar 30, 2022
Update sonic-utilities submodule to include below commits:
510a9b9 Add date to show version (#2086)
61b1396 [sfpshow][recycle_port] sfpshow script needs to skip recycle ports (#2109)
4681697 Support command `show ip fib` and `show ipv6 fib` (#2100)
2a982a1 Update `config mirror_session` CLI to support heximal gre type value (#2095)
b25f1e1 [generic-config-updater] Add caclrule validator (#2103)
968900c [sonic-package-manager] do not mod_config for whole config db when setting init_cfg (#2055)
bf55ceb [nvgre] Added auto-generated CLI plugins and test for NVGRE Tunnel feature (#1915)
8389c81 [sonic-cli-gen] fix failure "Error: digits_class" when field "digit_class" does not exist in DB (#2054)
f71ef64 Fix import statement in mclag.py (#2073)
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
…tting init_cfg (sonic-net#2055)

What I did
conn.mod_config(new_cfg) modifies entries which where not changed in new_cfg.
This causes errors when installing/upgrading extension like, because mod_config pushes the same configuration again:

"ERR swss#orchagent: :- addOperation: Vxlan tunnel 'tunnel1' is already exists” in syslog
I made a change so that when setting init configuration for an extension we are not modifying the whole DB.

How I did it
Perform a conn.mod_entry for entries from init configuration of the package.

How to verify it
Run UT. Make sure no errors when installing extension and having vxlan configuration in config db.
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants